Skip to content

fix: add underscore prefix to collection directories for Jekyll - #41

Merged
tallnato merged 1 commit into
mainfrom
fix/nat-90-category-collection-items
Mar 26, 2026
Merged

fix: add underscore prefix to collection directories for Jekyll#41
tallnato merged 1 commit into
mainfrom
fix/nat-90-category-collection-items

Conversation

@tallnato

Copy link
Copy Markdown
Member

Summary

  • Collection directories under content/ were missing the underscore prefix that Jekyll requires when using collections_dir
  • Without the prefix, site[page.collection_name] returns empty, causing category pages to show no items
  • Renamed all 5 collection dirs to _locksounds, _lightshows, _boombox, _wraps, _hornsounds
  • Updated scripts/build.ts and PR templates to match

Context

Follow-up to PR #40 which correctly switched the template to use site[page.collection_name], but the directories weren't properly named for Jekyll to recognize them as collections.

Test plan

  • Visit each category page (e.g., /Volts/locksounds/) and verify items appear
  • Click through to item detail pages
  • Run npx ts-node scripts/build.ts and verify JSON output is correct

🤖 Generated with Claude Code

Jekyll requires collection directories under collections_dir to be
prefixed with underscore (e.g., content/_locksounds/ not content/locksounds/).
Without the prefix, Jekyll treats the markdown files as regular pages
instead of collection documents, causing site[page.collection_name]
to return empty and category pages to show no items.

- Renamed all 5 collection dirs: _locksounds, _lightshows, _boombox, _wraps, _hornsounds
- Updated scripts/build.ts to reference underscore-prefixed dirs
- Updated PR templates with correct directory paths

Co-Authored-By: Paperclip <noreply@paperclip.ing>
@tallnato
tallnato force-pushed the fix/nat-90-category-collection-items branch from 764efc2 to 03e4ce5 Compare March 26, 2026 18:08
@tallnato
tallnato merged commit 201c656 into main Mar 26, 2026
2 checks passed
@tallnato
tallnato deleted the fix/nat-90-category-collection-items branch March 26, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant